Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compartmentalize README doc/shell tests #261

Merged
merged 3 commits into from
Nov 14, 2022
Merged

Conversation

bskinn
Copy link
Owner

@bskinn bskinn commented Nov 14, 2022

With this PR, they're no longer run as part of the core test suite; only as part of the Azure Pipelines auxiliary CI.

This will help ease the pain of #260, and also will minimize the thrashing of the commit history during general development... README glitches will only be surfaced on PR merge and releases.

Closes #260.

They're now no longer run as part of the core test suite; only
as part of the Azure Pipelines auxiliary CI.

This will help ease the pain of #260, and also will minimize
the thrashing of the commit history during general development...
README glitches will only be surfaced on PR merge and releases.

Closes #260.
@codecov-commenter
Copy link

codecov-commenter commented Nov 14, 2022

Codecov Report

Merging #261 (3f5351e) into main (312b178) will decrease coverage by 0.13%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##              main     #261      +/-   ##
===========================================
- Coverage   100.00%   99.86%   -0.14%     
===========================================
  Files           20       20              
  Lines          737      737              
===========================================
- Hits           737      736       -1     
- Misses           0        1       +1     
Impacted Files Coverage Δ
src/sphobjinv/cli/load.py 98.68% <0.00%> (-1.32%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@bskinn bskinn merged commit 1cce07b into main Nov 14, 2022
@bskinn bskinn deleted the readme-tests-default-deselect branch November 14, 2022 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2.3: pytest is failing in two inits
2 participants